Use umbrellas instead of direct includes in nativemodule/core module (#58711) - #58711

…eact#58711)

Summary:

Changelog: [Internal]

Part of the C++ stable API migration to umbrella includes. The `nativemodule/core` headers included other public modules through their fine-grained, `UmbrellaGuard`-protected headers. They now include those modules' umbrellas instead: `<React/CallInvoker.h>`, `<React/Bridging.h>` and `<React/RuntimeExecutor.h>`.

This matters most for the iOS `RCTTurboModule*.h` headers, which are exported, have no guard, and are not reached through `<React/NativeModuleCore.h>`. Includes of the module's own headers and all `.cpp`/`.mm` files are unchanged. No functional change.

Differential Revision: D122123056

…eact#58711)

Summary:

Changelog: [Internal]

Part of the C++ stable API migration to umbrella includes. The `nativemodule/core` headers included other public modules through their fine-grained, `UmbrellaGuard`-protected headers. They now include those modules' umbrellas instead: `<React/CallInvoker.h>`, `<React/Bridging.h>` and `<React/RuntimeExecutor.h>`.

This matters most for the iOS `RCTTurboModule*.h` headers, which are exported, have no guard, and are not reached through `<React/NativeModuleCore.h>`. Includes of the module's own headers and all `.cpp`/`.mm` files are unchanged. No functional change.

Differential Revision: D122123056

…eact#58711)

Summary:
Pull Request resolved: react#58711

Changelog: [Internal]

Part of the C++ stable API migration to umbrella includes. The `nativemodule/core` headers included other public modules through their fine-grained, `UmbrellaGuard`-protected headers. They now include those modules' umbrellas instead: `<React/CallInvoker.h>`, `<React/Bridging.h>` and `<React/RuntimeExecutor.h>`.

This matters most for the iOS `RCTTurboModule*.h` headers, which are exported, have no guard, and are not reached through `<React/NativeModuleCore.h>`. Includes of the module's own headers and all `.cpp`/`.mm` files are unchanged. No functional change.

Differential Revision: D122123056

…eact#58711)

Summary:
Pull Request resolved: react#58711

Changelog: [Internal]

Part of the C++ stable API migration to umbrella includes. The `nativemodule/core` headers included other public modules through their fine-grained, `UmbrellaGuard`-protected headers. They now include those modules' umbrellas instead: `<React/CallInvoker.h>`, `<React/Bridging.h>` and `<React/RuntimeExecutor.h>`.

This matters most for the iOS `RCTTurboModule*.h` headers, which are exported, have no guard, and are not reached through `<React/NativeModuleCore.h>`. Includes of the module's own headers and all `.cpp`/`.mm` files are unchanged. No functional change.

Differential Revision: D122123056

…eact#58711)

Summary:
Pull Request resolved: react#58711

Changelog: [Internal]

Part of the C++ stable API migration to umbrella includes. The `nativemodule/core` headers included other public modules through their fine-grained, `UmbrellaGuard`-protected headers. They now include those modules' umbrellas instead: `<React/CallInvoker.h>`, `<React/Bridging.h>` and `<React/RuntimeExecutor.h>`.

This matters most for the iOS `RCTTurboModule*.h` headers, which are exported, have no guard, and are not reached through `<React/NativeModuleCore.h>`. Includes of the module's own headers and all `.cpp`/`.mm` files are unchanged. No functional change.

Differential Revision: D122123056

…eact#58711)

Summary:

Changelog: [Internal]

Part of the C++ stable API migration to umbrella includes. The `nativemodule/core` headers included other public modules through their fine-grained, `UmbrellaGuard`-protected headers. They now include those modules' umbrellas instead: `<React/CallInvoker.h>`, `<React/Bridging.h>` and `<React/RuntimeExecutor.h>`.

This matters most for the iOS `RCTTurboModule*.h` headers, which are exported, have no guard, and are not reached through `<React/NativeModuleCore.h>`. Includes of the module's own headers and all `.cpp`/`.mm` files are unchanged. No functional change.

Reviewed By: cortinico

Differential Revision: D122123056

…cies (#58845)

Summary:
The iOS `react-native-clipboard/clipboard` job in the [nightly tests](https://github.com/react-native-community/nightly-tests/actions/runs/37195709435/job/111417053963) has failed since the `20261003` nightly:

```
Pods/Headers/Public/ReactNativeDependencies/folly/Expected.h:1587:10:
fatal error: 'folly/coro/Coroutine.h' file not found
```

Since #58711, `TurboModule.h` includes the `React/Bridging.h` umbrella, which includes `Dynamic.h` and therefore `folly/dynamic.h`. So every library that includes `RCTTurboModule.h` (for example through its codegen spec header) now compiles folly headers.

Some libraries set their own folly flags. Clipboard, for example, passes `-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1`. With `FOLLY_NO_CONFIG`, `folly-config.h` is skipped and `FOLLY_CFG_NO_COROUTINES` is never defined. Under C++20, folly then turns on coroutines and includes `folly/coro/Coroutine.h`, which the prebuilt `ReactNativeDependencies` doesn't ship.

This change makes `install_modules_dependencies` always append `-DFOLLY_CFG_NO_COROUTINES=1` to the library's compiler flags. React Native's own pods already use this flag (see `Helpers::Constants.folly_config`).

## Changelog:

[IOS] [FIXED] - Fix `'folly/coro/Coroutine.h' file not found` in libraries that set their own folly compiler flags

Pull Request resolved: #58845

Test Plan:
- Updated `new_architecture-test.rb` to assert that `spec.compiler_flags` contains `-DFOLLY_CFG_NO_COROUTINES=1`, both when new arch is enabled and when it is disabled.
- Ran the CocoaPods Ruby tests locally, excluding `spm-test.rb` because the `cocoapods` gem isn't available outside bundler in my environment: `ruby -Itest all_tests.rb -n /installModulesDependencies/`. The new `compiler_flags` assertions pass. In my local Ruby 3.4 environment the same 28 tests fail on both `main` and this branch, including a `HEADER_SEARCH_PATHS` assertion right after the new ones. That's an environment issue, not this change; relying on CI for the full suite.
- I didn't run the nightly-tests clipboard job against this change. It should pass once a nightly includes this fix.

Reviewed By: vzaidman

Differential Revision: D123393629

Pulled By: cipolleschi

fbshipit-source-id: b0ce442e9ded7842e1137b8379efb021f36d9919